Skip to content

Conversation

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM @gewarren

Let's :shipit:

@gewarren gewarren merged commit 4f20ce9 into dotnet:main Apr 29, 2025
22 checks passed
@gewarren gewarren deleted the warnings-april25 branch April 29, 2025 15:25
Copilot AI added a commit that referenced this pull request Nov 4, 2025
The note incorrectly stated that TargetFrameworks property is ignored if TargetFramework is specified. The actual behavior is the opposite - when both properties are present, TargetFrameworks (plural) takes precedence and TargetFramework (singular) is ignored.

Updated the note to correctly reflect this behavior.

Fixes #45942

Co-authored-by: gewarren <[email protected]>
gewarren added a commit that referenced this pull request Nov 4, 2025
)

* Initial plan

* Fix TargetFrameworks documentation note

The note incorrectly stated that TargetFrameworks property is ignored if TargetFramework is specified. The actual behavior is the opposite - when both properties are present, TargetFrameworks (plural) takes precedence and TargetFramework (singular) is ignored.

Updated the note to correctly reflect this behavior.

Fixes #45942

Co-authored-by: gewarren <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: gewarren <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants